Skip to content

Conversation

@jdkfx
Copy link
Contributor

@jdkfx jdkfx commented Mar 21, 2025

内容

以下のPRからCherry-pickして少し修正を追加

関連 Issue

#2011

スクリーンショット・動画など

その他

@jdkfx jdkfx requested a review from a team as a code owner March 21, 2025 14:42
@jdkfx jdkfx requested review from Hiroshiba and removed request for a team March 21, 2025 14:42
@jdkfx jdkfx changed the title Refactor/improve e2e test エディター選択ダイアログに関するE2Eテストを追加 Mar 21, 2025
@voicevox-preview-pages
Copy link

voicevox-preview-pages bot commented Mar 21, 2025

🚀 プレビュー用ページを作成しました 🚀

更新時点でのコミットハッシュ:910eefd

@Hiroshiba
Copy link
Member

Hiroshiba commented Mar 21, 2025

あ、主旨が違うかもです!

やりたいのは #2409 のプルリクエストの一部を切り分けることなので、こっちをマージしてから #2409 をマージするのが良いと思います。
#2409 がエディター選択ダイアログを作ることと、e2eテスト周りのリファクタリングを含んでいるので、リファクタリングだけ先に切り出す感じ!

なのでタイトルは「refactor: e2eテストでのトーク・ソング画面へのnavigate関数を分離」とかにして、navigateToTalknavigateToSongを今のmainブランチに合う形にするのが良いのかなと!
これで #2409 のテストに合わせるための変更が最小で済む・・・はず!

@jdkfx
Copy link
Contributor Author

jdkfx commented Mar 22, 2025

@Hiroshiba ありがとうございます!タイトルそのままいただいて変更させていただきます!

@jdkfx jdkfx changed the title エディター選択ダイアログに関するE2Eテストを追加 refactor: e2eテストでのトーク・ソング画面へのnavigate関数を分離 Mar 22, 2025
@jdkfx jdkfx marked this pull request as draft January 15, 2026 15:15
@jdkfx
Copy link
Contributor Author

jdkfx commented Jan 15, 2026

すみません、あまりにも放置しすぎていろいろと変わっていると思いますので、クローズしますね。

@jdkfx jdkfx closed this Jan 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants